Skip to content

Pass node_rank instead of rank when computing the engine launch command - #2003

Open
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-51from
tom/refactor-miles/op13-48
Open

Pass node_rank instead of rank when computing the engine launch command#2003
fzyzcjy wants to merge 1 commit into
tom/refactor-miles/op13-51from
tom/refactor-miles/op13-48

Conversation

@fzyzcjy

@fzyzcjy fzyzcjy commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Part of #1837

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Caution

The consumer version of Gemini Code Assist on GitHub has been sunset. All code review activity has officially ceased.

@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 394e00b to 6fd1deb Compare August 2, 2026 12:40
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch 3 times, most recently from 9f114ae to b45bd94 Compare August 4, 2026 00:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 37bcb24 to a0f6fb2 Compare August 4, 2026 00:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from b45bd94 to 49585ab Compare August 4, 2026 06:54
@fzyzcjy
fzyzcjy requested a review from Rockdu as a code owner August 4, 2026 06:54
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from a0f6fb2 to e112a45 Compare August 4, 2026 06:54
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 49585ab to a2ae9e9 Compare August 4, 2026 07:26
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from e112a45 to 4915e35 Compare August 4, 2026 07:26
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from a2ae9e9 to a45e897 Compare August 4, 2026 07:33
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 4915e35 to 9a7e1a4 Compare August 4, 2026 07:33
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from a45e897 to 939d1ba Compare August 6, 2026 03:48
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch 2 times, most recently from 92b8cfb to 9f8b0ba Compare August 6, 2026 07:04
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 939d1ba to 9778253 Compare August 6, 2026 07:04
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 9f8b0ba to 68cdef2 Compare August 9, 2026 23:32
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch 2 times, most recently from 442059f to ae7dfac Compare August 10, 2026 00:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 68cdef2 to a39af60 Compare August 10, 2026 00:44
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from ae7dfac to 80865e8 Compare August 10, 2026 06:03
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 6e19c4c to a978a5d Compare August 13, 2026 03:11
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 04d842a to 9f4cf59 Compare August 13, 2026 03:11
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from a978a5d to 40c3afb Compare August 13, 2026 03:25
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch 3 times, most recently from d0ec1f2 to b711af0 Compare August 13, 2026 06:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 121a3cd to 7d4b538 Compare August 13, 2026 06:39
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from b711af0 to b4d8f53 Compare August 14, 2026 04:46

@guapisolo guapisolo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Stack-verified: #2063 removes the obsolete global-rank alignment guard, and #2096 repairs the stale test caller and adds cross-cell coverage. Both fixes land before #2176 and remain intact through #2649.

@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 29b9698 to 92be36b Compare August 29, 2026 12:38
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from b4d8f53 to 7b1653c Compare August 29, 2026 12:38
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 92be36b to 5ee39fe Compare August 31, 2026 23:34
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch 2 times, most recently from 4fe0608 to c1a11de Compare September 1, 2026 10:03
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch 2 times, most recently from 934b09f to 4278621 Compare September 1, 2026 10:36
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from c1a11de to 2a515cb Compare September 1, 2026 10:36
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 4278621 to aa58344 Compare September 1, 2026 14:33
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 2a515cb to b652669 Compare September 1, 2026 14:33
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from aa58344 to dcd145d Compare September 3, 2026 06:02
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from b652669 to 1666761 Compare September 3, 2026 06:02
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from dcd145d to 3e949cd Compare September 3, 2026 06:21
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 1666761 to 1b9ea12 Compare September 3, 2026 06:21
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 3e949cd to becc1d2 Compare September 3, 2026 06:33
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 1b9ea12 to 1244267 Compare September 3, 2026 06:33
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from becc1d2 to 456eca7 Compare September 3, 2026 07:53
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 1244267 to 1c90488 Compare September 3, 2026 07:53
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-51 branch from 456eca7 to dd763eb Compare September 3, 2026 10:52
@fzyzcjy
fzyzcjy force-pushed the tom/refactor-miles/op13-48 branch from 1c90488 to 1f09d4e Compare September 3, 2026 10:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants